-
Notifications
You must be signed in to change notification settings - Fork 160
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mise à jour des instructions d'installation pour macOS. #6427
Mise à jour des instructions d'installation pour macOS. #6427
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Belle documentation !
Peut-être un peu trop détaillée par endroits, mais je ne vais pas te faire retirer des phrases :)
Je suis étonné que tu ais eu besoin d'installer les coreutils GNU/Linux. Sur mon Mac bloqué à la version BigSur 11.6, je ne me souviens pas précisément comment j'avais installé initialement ZdS, mais je n'avais pas eu besoin de faire ces manipulations...
Quelques typos, problèmes de formatage et reformulations suggérées et on pourra merger.
C'est OK pour moi, j'ai repris toutes les remarques, corrigé les apostrophes, renommé le fichier afin d'être cohérent avec le nouveau nom de macOS, et voilà ! |
J'avais pas vu certains commentaires de @philippemilink.
J'ai voulu rendre la documentation accessible y compris à des débutant·es complets, afin de faciliter la contribution. C'est pour ça que la documentation s'apparente presque à un tutoriel à certains endroits. C'est aussi pour ça que j'ai pensé à deux niveaux de lecture, en écrivant un chemin de contournement.
C'est probablement faisable en contournant le script d'installation et en faisant les étapes à la main (comme la CI), ou en réécrivant le script pour qu'il soit aussi compatible avec BSD/Darwin. J'ai fait ce choix afin de continuer d'utiliser le script d'installation, et donc d'être futur-proof, évitant que la documentation pour macOS ne se décale par manque de mise à jour. On peut faire autrement, donc, mais c'est plus d'effort. |
Co-authored-by: Ph. SW. <philippe.milink@gmx.fr>
Pour éviter que Sphinx ne fasse n'importe quoi.
5f903ab
to
2d28e0f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je n'ai pas testé (j'ai déjà ZdS installé sur mon Mac), mais ça me semble correct. QA OK ✔️
…ussi-droit-dinstaller-zds
Mise à jour majeure de la documentation d'installation sous macOS 12+.
Cette mise à jour comprend :
Makefile
;Contrôle qualité
Le commit n'est pas signé car je n'ai pas encore reconfiguré la signature automatique de mes commits sur mon nouvel ordi — sous macOS, vous l'aurez deviné. Ayant configuré GitHub pour exiger la signature de mes commits, c'est normal qu'il râle. C'est bien moi :p .